-
-
Notifications
You must be signed in to change notification settings - Fork 591
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set minimum supported Matrix 1.1 version (drop legacy r0 versions) #3007
Conversation
@turt2live is this still ongoing? |
Not by me, but is increasingly critical work the team will need to pick up. |
All the Prerequisites are merged, and the code looks fine to me. The description says WIP - is there more to do @turt2live ? |
I don't recall, sorry. This will need investigation from the team. |
No problem. I don't see value in leaving this open in draft form since we are not currently working on it. The code will be valuable when we get to it, thank you. |
…avis/mx-v1.1/api # Conflicts: # src/autodiscovery.ts
…avis/mx-v1.1/api # Conflicts: # src/client.ts
this PR should also probably update the README text to say there's now a minimum version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clearing my own review as I have taken over
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Co-authored-by: Germain <germains@element.io>
WIP
Fixes element-hq/element-web#16876
Requires matrix-org/matrix-react-sdk#9819
Prerequisites:
device_id
to/account/whoami
types #3005m.key.verification.ready
event type definition #3006Here's what your changelog entry will look like:
🚨 BREAKING CHANGES